Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix importing files with dots #20

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

eppsilon
Copy link

This PR was forked from @JohnAlbin's branch from #19. It would probably be best to merge that before reviewing this, or you can just look at commit 8025bbc.

@TheSharpieOne
Copy link

lgtm... Any movement on this.

@Snugug
Copy link
Member

Snugug commented Feb 21, 2016

Merge conflicts :(

@eppsilon eppsilon force-pushed the fix-importing-files-with-dots branch from 8025bbc to b158799 Compare February 22, 2016 23:17
@eppsilon
Copy link
Author

I updated the branch with the following changes:

  • Rebased atop this repo's master
  • Removed .only call in tests file - that was the only test being run (!!!)
  • Disabled some eslint warnings about code that was already there

@ivanvoznyakovsky
Copy link

Will this get merged?

@eppsilon
Copy link
Author

I created this PR in November and rebased it in February, so...not looking likely. @Snugug ?

@0x6a68
Copy link

0x6a68 commented Jan 9, 2017

@Snugug happy new year! could this be merged some how? or do you want to have more simplified PR on this topic?

@AndyOGo
Copy link

AndyOGo commented Feb 20, 2017

@eppsilon @TheSharpieOne @ivanvoznyakovsky @johannestroeger
This repos seems to be unmaintained, hence we create a fork and republished it on npm as @axa-ch/node-sass-import-once
In case you need this still to be merged, just open a PR at https://github.com/axa-ch/node-sass-import-once ... we will be happy to improve this valueable piece of code 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants